Skip to content

213#214

Merged
RAprogramm merged 1 commit intomainfrom
213
Oct 17, 2025
Merged

213#214
RAprogramm merged 1 commit intomainfrom
213

Conversation

@RAprogramm
Copy link
Owner

Summary

Fixed Codecov unknown coverage and test analytics configuration by separating responsibilities between coverage and test jobs, matching the working configuration from infra-metrics-insight-renderer.

Changes

Coverage Job

  • Removed id-token: write permission (not needed with token-based auth)
  • Removed codecov/test-results-action (moved to test job)
  • Removed JUnit XML generation step (handled by test job)
  • Simplified summary output

Test Job

  • Added cargo-nextest installation for MSRV version
  • Split test execution: nextest with ci profile for MSRV, regular cargo test for stable
  • Added codecov/test-results-action for MSRV test results upload
  • Removed redundant JSON test report generation

cargo-test Action

  • Added nextest-profile parameter for conditional nextest execution
  • Maintains backward compatibility with regular cargo test

Technical Details

JUnit XML Generation:

  • Generated via nextest ci profile: target/nextest/ci/junit.xml
  • Auto-discovered by codecov/test-results-action

Authentication:

  • Token-based only: token: ${{ secrets.CODECOV_TOKEN }}
  • No OIDC conflicts

Benefits

  • Coverage and test analytics properly separated
  • No authentication conflicts
  • Cleaner job responsibilities
  • Consistent with working project architecture

Test Plan

  • All 236 tests pass locally with nextest ci profile
  • JUnit XML generated successfully (43KB)
  • YAML validation passed
  • Pre-commit hooks passed

Closes #213

…ht-renderer.

Changes:
- Coverage job: removed id-token permission and test-results-action
- Test job: added nextest with ci profile for MSRV, uploads test results
- cargo-test action: added nextest-profile parameter for JUnit XML generation
- Removed redundant JSON test report generation and artifact uploads
- Simplified summary outputs for better clarity

Benefits:
- Coverage and test analytics properly separated
- Token-based authentication only (no OIDC conflicts)
- JUnit XML auto-generated via nextest ci profile
- Cleaner job responsibilities and artifact management

Test plan:
- All 236 tests pass with nextest ci profile
- JUnit XML generated at target/nextest/ci/junit.xml (43KB)
- YAML validation successful

Closes #213
@RAprogramm RAprogramm merged commit c2de1ce into main Oct 17, 2025
16 checks passed
@RAprogramm RAprogramm deleted the 213 branch October 17, 2025 05:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix Codecov unknown coverage and test analytics configuration

1 participant